Skip to content

Conversation

wangmengyan95
Copy link
Contributor

In order to expose the raw response(ungziped response) to developers, we need to disable the transparent ungzip for ApacheHttpClient and URLConnection and add a decompress interceptor at the beginning of the interceptor list manually.
For OkHttp, since we can not disable the transparent ungzip, we will handle it separately.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: newline at EOF

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kk

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: whitespace

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kk

@grantland grantland assigned wangmengyan95 and unassigned grantland Aug 25, 2015
@wangmengyan95 wangmengyan95 force-pushed the wangmengyan.add_decompress_interceptor_when_we_have_external_interceptors branch from 79d0f96 to 6802178 Compare August 27, 2015 20:25
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment somewhere why we set Content-Length and totalSize to -1?

@grantland
Copy link
Contributor

Small nits, squash and LGTM

@wangmengyan95 wangmengyan95 force-pushed the wangmengyan.add_decompress_interceptor_when_we_have_external_interceptors branch from 6802178 to 01a30d5 Compare August 28, 2015 18:29
wangmengyan95 added a commit that referenced this pull request Aug 28, 2015
…interceptor_when_we_have_external_interceptors

Disable transparent ungzip and add decompress interceptor
@wangmengyan95 wangmengyan95 merged commit 765589c into master Aug 28, 2015
@wangmengyan95 wangmengyan95 deleted the wangmengyan.add_decompress_interceptor_when_we_have_external_interceptors branch August 28, 2015 18:32
@facebook-github-bot
Copy link

@wangmengyan95 updated the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants